Skip to content

Add a dispatchable workflow to set npm dist-tags - #151

Merged
luiz-lvj merged 1 commit into
masterfrom
chore/npm-dist-tag-workflow
Aug 21, 2026
Merged

Add a dispatchable workflow to set npm dist-tags#151
luiz-lvj merged 1 commit into
masterfrom
chore/npm-dist-tag-workflow

Conversation

@luiz-lvj

Copy link
Copy Markdown
Collaborator

Add a dispatchable workflow to set npm dist-tags

@luiz-lvj
luiz-lvj requested review from pepebndc and tirumerla August 21, 2026 17:21

@tirumerla tirumerla left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a comment

Comment thread .github/workflows/npm-dist-tag.yml
@luiz-lvj
luiz-lvj merged commit 455cf29 into master Aug 21, 2026
15 checks passed
luiz-lvj added a commit that referenced this pull request Aug 24, 2026
* add a dispatchable workflow to set npm dist-tags (#151)

* Refresh package-lock.json for the moved hardhat-tron main

The @openzeppelin/hardhat-tron git dependency tracks main. Their main
now consumes @openzeppelin/tron-runtime ^0.1.0 from npm and bumps
tronweb to 6.5.0, so npm ci fails against the stale lockfile on every
branch. Re-resolve the lockfile. The jar-patch source pin follows the
lockfile, so TRE jars now build from hardhat-tron 603106cd.

Validated locally: clean npm ci, jar rebuild via build-tre-fork.sh
(tre_version reports v1.0.4-oz-tron), tron-solc compile, and TRE smoke
tests (P256 and Create2 suites) against the rebuilt jar.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* Fix the toBeArray call broken by the ethers 6.17 width parameter

ethers 6.17 adds a width parameter to toBeArray. Array.map passes the
element index as the second argument, so .map(ethers.toBeArray) now
throws a NUMERIC_FAULT overflow for width 0. Call it with the value
only, matching upstream openzeppelin-contracts.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Luiz Vasconcelos Júnior <64055364+luiz-lvj@users.noreply.github.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Luiz <luiz.vasconcelos@openzeppelin.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants